Skip to content

Conversation

@flyingrobots
Copy link
Member

@flyingrobots flyingrobots commented Feb 6, 2026

Summary

  • add fuzz/junk JSONL command channel test
  • add alfredctl integration test
  • update alfred-live changelog

Testing

  • pnpm test

Summary by CodeRabbit

  • Tests
    • Added integration tests validating CLI JSONL output and end-to-end envelope handling.
    • Added fuzz-style tests to ensure malformed or junk input is handled without errors.
  • Chores
    • Bumped package versions to 0.10.1 and updated changelogs.
  • Documentation
    • Updated README latest release entry to v0.10.1.

@coderabbitai
Copy link

coderabbitai bot commented Feb 6, 2026

📝 Walkthrough

Walkthrough

Adds test coverage: a fuzz-style randomized-input test for the JSONL command channel and an integration test that validates alfredctl CLI JSONL output; includes test utilities and version/changelog/package bumps across alfried and alfried-live packages.

Changes

Cohort / File(s) Summary
Changelog / Release Metadata
alfred-live/CHANGELOG.md, alfred/CHANGELOG.md, CHANGELOG.md, alfred-live/jsr.json, alfred/jsr.json, alfred-live/package.json, alfred/package.json, README.md
Adds Unreleased / 0.10.1 entries and updates latest release; bumps versions from 0.10.0 → 0.10.1 in package metadata and updates dependency version; README release line adjusted.
Tests (new)
alfred-live/test/unit/alfredctl.test.js, alfred-live/test/unit/command-envelope.test.js
Adds an integration test for alfredctl producing JSONL envelopes and a fuzz-style unit test that feeds randomized/junk JSONL lines through the command envelope pipeline; includes deterministic PRNG and helper generators.
Repository Config
.npmrc
Adds link-workspace-packages=true to npm workspace config.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 I hopped through lines of JSONL bright,
Poked at junk and random bytes,
The CLI sang its envelope tune,
Tests now guard the afternoon,
🥕 Hooray — the checks pass light!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title directly and accurately summarizes the main changes: adding JSONL fuzz testing and alfredctl integration tests to alfred-live.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/v0.10.1-command-channel

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@flyingrobots
Copy link
Member Author

@coderabbitai review please

@coderabbitai
Copy link

coderabbitai bot commented Feb 6, 2026

@flyingrobots: I'll review the changes in this PR for you.

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@flyingrobots flyingrobots merged commit 9e94753 into main Feb 6, 2026
2 checks passed
@flyingrobots flyingrobots deleted the feat/v0.10.1-command-channel branch February 6, 2026 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant